Conversation
Can now set the names of the Moller collections, to differentiate between GBL and KF tracks being used.
Detector based on the HPS_Run2021Pass1Top detector used at 3.7GeV running
cbravo135
left a comment
There was a problem hiding this comment.
I have been using this branch for months, given multiple talks using it, including the recent work I did aligning the 2021 SVT.
|
Has the full MC chain been checked with this on a few of the new detectors and maybe a couple of the old ones? Might want to do this as it would verify that the LCDD and compact don't have a geometry mismatch. |
|
I don't think it is needed to check the MC chain for multiple detectors for this PR. The change that would potentially change stuff wrt the lcdd/compact already happened in #903. The one thing here that I didn't realize was in the branch are changes from TongTong for the DAQ configuration work he did. I do have some comments on how that was done, mostly that there is a big block of else if statements to choose the correct resource for the correct run number. This should have been done using the database, as this sort of issue is exactly what the database system was designed to solve. |
For a DAQ configuration, it includes some parameters, but not much like gain or pedestals or else for Ecal/Hodo/SVT channels. |
| return "hps_v12_1"; | ||
| else if(runNumber >= 10716 && runNumber <= 10718) | ||
| return "hps_v13_FEE"; | ||
|
|
There was a problem hiding this comment.
Just to be clear for TongTong, my comment was not about the data from the config files being a resource. My comment was about this particular block of code being a mess. What this code achieves can be more cleanly achieved via the db, by making a table that holds the names of each of the config file resources that could be used and making entries in the conditions db to associate each one with the appropriate run numbers.
There was a problem hiding this comment.
It sounds not a big job. Let's put it into to-do list.
The current code works. I suggest to merge the branch firstly, and then open an issue to do update later.
tongtongcao
left a comment
There was a problem hiding this comment.
I worked on the branch for a while. No issues happened. So I approve the PR.
Time to bring back run2021 into master.